-
Notifications
You must be signed in to change notification settings - Fork 300
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
マルチトラック:mainにマージ #2185
Merged
Merged
マルチトラック:mainにマージ #2185
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Change: phrase以外をtrackIdを受け取るようにする * wip * Update: singing.tsを色々変える * Update: store以外を良い感じに合わせる * Add: マイグレーションを追加 * Fix: SET_TRACKでselectedTrackIdが無を参照するのを修正 * Fix: 二重にセットするように * Fix: trackOrderにする * Change: SELECT_TRACKしないようにする * Code: data -> pitchArray * Code: SET_SINGING_GUIDE_KEY_TO_PHRASEに型をつける * Code: こっちも型をつける * WIP: むずい * Change: PhraseにtrackIdを持たせる * Fix: 型エラーを修正 * Change: SET_NOTESとかの処理をSET_TRACKに写す * Fix: overlappingNoteIds.hasが抜けてたのを修正 * Change: initialTrackIdにする * Code: コメントを修正 * Change: singerAndFrameRatesを最初に作っておく * Code: トラック周りのactionを移動 * Fix: overlappingNoteInfos周りを修正 * Change: Mapのキーを変える Co-Authored-By: Hiroshiba <Hiroshiba@users.noreply.github.com> * Change: CREATE_TRACKをADD_TRACKと分割 * Code: 空行を開ける Co-Authored-By: Hiroshiba <Hiroshiba@users.noreply.github.com> * Change: overlappingNoteIdsをトラック毎に持つようにする * Code: コメントを移動 Co-Authored-By: sigprogramming <sigprogramming@users.noreply.github.com> * Fix: Renderする * Add: trackOrderに入ってるかバリデーションする * Add: tracksに入ってるかバリデーションする * Delete: SET_TRACK_ORDERを一度消す * Delete: singerAndFrameRatesの片引き数を消す * Add: trackのバリデーションを追加 * Fix: overlappingNoteIdsを代入してなかったので修正 * Change: 判定方法を変更 * Change: tracks.hasに一本化する --------- Co-authored-by: Hiroshiba <Hiroshiba@users.noreply.github.com> Co-authored-by: sigprogramming <sigprogramming@users.noreply.github.com>
## 内容 project-multitrackのbufferにmainをマージします。 ## 関連 Issue (なし) ## スクリーンショット・動画など (なし) ## その他 (なし)
* Refactor: コンポーネントを分離 * Fix: エンジンアイコン周りの変更に追従 * Fix: engineIconsの引数を修正 * Apply suggestions from code review --------- Co-authored-by: Hiroshiba <hihokaruta@gmail.com>
* Change: SET_TRACKを作る * Add: インポートをマルチトラック対応 * Add: 複数トラックをインポートできるように * Fix: indexじゃなくてiだった * Add: 共通化 + undo出来るように * Change: トラック選択のUIを変更 * Change: mapを使う * Change: isTracksEmptyを移動 * Change: IMPORT_*_PROJECTをCOMMANDに * Add: isValidTrackチェックを追加 * Change: QCheckboxを使う * Change: 見た目を良い感じにする * デザインちょっと調整 --------- Co-authored-by: Hiroshiba <hihokaruta@gmail.com>
## 内容 mainをマルチトラックのブランチにマージします。 ## 関連 Issue (なし) ## スクリーンショット・動画など (なし) ## その他 (なし)
## 内容 project-multitrackにmainを持ってきます。主に #2128 です。 ## 関連 Issue (なし) ## スクリーンショット・動画など (なし) ## その他 conflictが無かったのでmainから直接投げています。一応buffer作った方が良いかも?
* Add: とりあえず移植 * Add: 型とかを追加 * Add: トラック追加できるように * Add: actionを追加 * Improve: 見た目を良い感じにする * Add: utaformatixの変換にnameを含める * Add: テストを追加 * Add: TODOを追加 * Delete: overlappingNoteInfosあたりを削除 * Add: フォールバックを追加 * Add: コメントを沢山追加 * Add: コメントをちょっと追加 * Add: コメントをもうちょっと追加 * Change: singer-name -> track-name * Delete: z-indexに頼らないようにする * Improve: CSS周りを整理 * Add: character周りをまとめる * add: cloneWithUnwrapProxyに包む * Add: 設定出来るように * Update: スナップショット更新 [update-snapshots] * Update: スナップショット更新 [update snapshots] * (スナップショットを更新) * Change: コメントの位置を変える * Change: QOptionGroupにする * Change: プロパティ名を変える * Refactor: 細かいところを良い感じにする Co-Authored-By: Hiroshiba <Hiroshiba@users.noreply.github.com> * Code: コメントを足す Co-Authored-By: Hiroshiba <Hiroshiba@users.noreply.github.com> * Code: コメントを足す * Delete: isDraggingを消す * Fix: 細かいところを修正 * (スナップショット更新) [update snapshots] * (スナップショットを更新) * Delete: TODOを消す Co-authored-by: Hiroshiba <hihokaruta@gmail.com> * Code: 順番を揃える Co-authored-by: Hiroshiba <hihokaruta@gmail.com> * Delete: $qを消す Co-authored-by: Hiroshiba <hihokaruta@gmail.com> * Change: volume -> gain * Change: singer-style -> singer-name * Code: コメントを追加 * Change: heightの指定をSingEditorに移す * Change: 影響を受けないようにするやり方を変える * Change: サイドバーの幅を保存する * Change: 幅を細かくする * Delete: 余白を消す * Change: content: strictに Co-authored-by: Hiroshiba <hihokaruta@gmail.com> * Change: watchEffectにする Co-authored-by: Hiroshiba <hihokaruta@gmail.com> * Delete: setTrackNameをなくす Co-authored-by: Hiroshiba <hihokaruta@gmail.com> * Change: コロンを使わないようにする Co-authored-by: Hiroshiba <hihokaruta@gmail.com> * Change: mutationの外でcloneWithUnwrapProxyを使う Co-authored-by: Hiroshiba <hihokaruta@gmail.com> * Add: watchEffectをインポートする * Change: 警告を出すのは開発時だけにする * Add: TODOを追加 * Change: 幅保存のロジックを変える Co-Authored-By: Hiroshiba <Hiroshiba@users.noreply.github.com> * Update src/components/Sing/SequencerPitch.vue --------- Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: Hiroshiba <Hiroshiba@users.noreply.github.com> Co-authored-by: Hiroshiba <hihokaruta@gmail.com>
## 内容 いつものアレです。 ## 関連 Issue - ref: #2163 ## スクリーンショット・動画など (なし) ## その他 (なし)
* Fix: stateの値に対してバリデーションをかける * Change: NOTEにする Co-authored-by: Hiroshiba <hihokaruta@gmail.com> * Code: コメントを良い感じにする Co-authored-by: Hiroshiba <hihokaruta@gmail.com> * Change: hasにする * Change: getSelectedTrackWithFallbackにする * Change: NOTE_IDS -> ALL_NOTE_IDS * Add: テストを追加 * Revert: 43fbc23. * Change: current -> selected * Change: SELECT_ALL_NOTES -> SELECT_ALL_NOTES_IN_SELECTED_TRACK * Change: トラック削除時は1つ上に飛ぶようにする * Code: コメントを良い感じにする * Fix: dispatchの名前を修正 * Apply suggestions from code review --------- Co-authored-by: Hiroshiba <hihokaruta@gmail.com>
* WIP * Add: とりあえず再生できるように * Add: 全書き出しをマルチトラックに対応 * Change: previewSynthをglobalChannelStripに直接繋ぐようにする * Change: generateDefaultSongFileNameをstoreの外に出す * Change: watchStoreStatePluginにする * Change: fileHelperから持ってくるようにする * Change: convertToWavFileDataを外に出す * Change: watchStoreStatePlugins -> watchStoreStatePlugin * Code: コメントを追加 * Change: RENDER内を良い感じにする * Delete: nextTickを削除 * Code: コメントの内容を変える * Fix: 2回設定していたのを削除 * Change: globalChannelStrip -> mainChannelStrip * Change: createAudioPlayerSequenceを戻す * Change: shouldPlayTracks内のフィルタに任せる * Change: getterにする * Add: limiterを追加
* Add: 他トラックのノートを出す様に * Add: PhraseIndicatorを追加 * Fix: z-indexを修正 * Add: SequencerPitchを対応 * Delete: z-indexに頼らないようにする * Code: コメントを動かす * Change: line -> shadow-line * Change: trackIdをnullableじゃなくする * Change: notes -> allNotes * Change: singingGuideKeysをフィルタする * Change: singingGuideをフィルタするようにする * Change: NoteにtrackIdを渡すのをやめる * Change: trackIdのキー毎削除 * Change: hasOverlappingの表示切り替えをコンポーネント側でやる Co-Authored-By: sigprogramming <sigprogramming@users.noreply.github.com> * Delete: singingGuidesを削除 Co-Authored-By: sigprogramming <sigprogramming@users.noreply.github.com> * Delete: 不要なnon null assertionを削除 Co-Authored-By: sigprogramming <sigprogramming@users.noreply.github.com> * Change: selectedNotes -> selectedNotesInCurrentTrack Co-Authored-By: sigprogramming <sigprogramming@users.noreply.github.com> * Code: コメントを修正 * Code: 細かいところを修正 * Change: hasOverlappingError -> isOverlapping Co-Authored-By: hiroshiba <hiroshiba@users.noreply.github.com> * Change: 順番を揃える Co-Authored-By: hiroshiba <hiroshiba@users.noreply.github.com> * Change: current -> selected Co-Authored-By: hiroshiba <hiroshiba@users.noreply.github.com> * Change: selectedNoteIdsInCurrentTrackを消す * Change: opacityを使わないようにする * Delete: 不要な変数を削除 * Delete: TODOを消す --------- Co-authored-by: sigprogramming <sigprogramming@users.noreply.github.com> Co-authored-by: hiroshiba <hiroshiba@users.noreply.github.com>
## 内容 main -> project-multitrackです。 ## 関連 Issue (なし) ## スクリーンショット・動画など (なし) ## その他 (なし)
* Change: shouldPlayTracksがSetを返すようにする * Change: 実験的機能に隔離 * Change: 条件を修正 * Code: コメントを追加 * Add: TODOを追加 ついでに[update snapshots] * (スナップショットを更新) * Change: トグルの変数を変える * Change: 抜け穴をふさぐ * Delete: セーフガードをなくす Co-authored-by: Hiroshiba <hihokaruta@gmail.com> * Change: solo/muteの代入方法を変える Co-authored-by: Hiroshiba <hihokaruta@gmail.com> * Change: CLEAR_UNDO_HISTORYにする * Change: disable周りを揃える * Change: number | nullにする * Change: findIndicesを消す * Change: テキストをほんのり変える * Fix: overlappingNoteIdsをコピーする * Change: RENDERでChannelStripを更新するようにする * Code: RENDERの前に空行を開ける * Fix: 合成状態の順序を修正 * Fix: ピッチが見えなかったのを修正 * Change: toSortedをかける * Change: SET_TRACKまわりを変える * Fix: ソートするものが間違ってた * Delete: watch周りのやつを削除 * Change: 変数名を変える * Delete: 使われてない型を削除 * Code: 注意を追加 * Code: コメントを変える * Code: コメントをいい感じにする Co-authored-by: Sig <sigprogrammer@gmail.com> * Change: SELECT_ALL_NOTES_IN_SELECTED_TRACK -> SELECT_ALL_NOTES_IN_TRACK * Fix: 挙動を揃える * Fix: ガバ * Fix: Render忘れ * Apply suggestions from code review --------- Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: Hiroshiba <hihokaruta@gmail.com> Co-authored-by: Sig <sigprogrammer@gmail.com>
## 内容 多分最後のconflict解消です。 ## 関連 Issue (なし) ## スクリーンショット・動画など (なし) ## その他 (なし)
Fix: 複数選択が出来ないのを修正
Hiroshiba
approved these changes
Jul 30, 2024
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!!
いったんお疲れ様でした!!!
マルチトラックを使っていろんな曲が作られると嬉しいですね!!
マルチトラック周りはまだまだ使いやすく&クールにできると思うので、どんどん改良できればと思います!!
その際はぜひよろしくお願いします!!
Comment on lines
-413
to
+435
export function selectPriorPhrase( | ||
phrases: Map<PhraseSourceHash, Phrase>, | ||
export function selectPriorPhrase<K extends string>( | ||
phrases: Map<K, Phrase>, | ||
position: number, | ||
): [PhraseSourceHash, Phrase] { | ||
): [K, Phrase] { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
この辺PhraseSourceHashだけに戻った気がするので、K extends stringじゃなくても良いかもですね
まあいいか。
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
内容
project-multitrackをmainにマージします。長く苦しい戦いだった(本当はまだ色々残ってるのでcloseじゃなくてrefにしてます)
関連 Issue
スクリーンショット・動画など
(なし)
その他
#2184 が先です。